Skip to content

deps: cherry-pick libuv/libuv@e640dc9 - #65118

Open
ulofiai wants to merge 1 commit into
nodejs:mainfrom
ulofiai:fix/issue-63638
Open

deps: cherry-pick libuv/libuv@e640dc9#65118
ulofiai wants to merge 1 commit into
nodejs:mainfrom
ulofiai:fix/issue-63638

Conversation

@ulofiai

@ulofiai ulofiai commented Aug 7, 2026

Copy link
Copy Markdown

Cherry-picks the upstream libuv fix for Windows fs-event watchers using 8.3 short paths.

When the watched directory is supplied in short-path form, resolving an event path to its long form can make it no longer share the stored directory prefix. Instead of asserting (or calculating an invalid relative path in release builds), fall back to the filename returned by ReadDirectoryChangesW, which is already relative to the watched directory. The regression test now creates a real directory with distinct long and short forms and tries both the Windows temporary directory and the current directory.

Fixes: #63638
Refs: libuv/libuv#5152
Refs: libuv/libuv@e640dc9

Signed-off-by: ulofiai <monsterking@tutamail.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added libuv Issues and PRs related to the libuv dependency or the uv binding. needs-ci PRs that need a full CI run. labels Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

libuv Issues and PRs related to the libuv dependency or the uv binding. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Potential regression with libuv in Node 24.16.0

2 participants